-
Notifications
You must be signed in to change notification settings - Fork 49
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
go to top modified #58
base: master
Are you sure you want to change the base?
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/ghulamyazdani/truhealth/8cvw8dGq8V9o9QYxfD1cibHgzbdc |
Hey @mandiraghosh783, there are some issues with the changes you made on the website. Due to the addition of the Back to Top button, Preview of ErrorExpected Result |
But, I have made no changes in index.html |
@mandiraghosh783 you have made changes in style.css file which is linked with index.html file as well. You may have changed the formatting in some classes which are linked to index.html file as well. So it will affect that file as well. |
ok, i will look into it once again. |
Yepp @mandiraghosh783, I have reviewed the changes and the current website is looking good. So, we will be soon merging this PR after resolving conflicts. @ghulamyazdani will resolve the conflicts on this PR. |
I have modified the go to top. You asked me to beautify the button more. I did that. Kindly check..